Skip to content

[backport cloud/1.41] Restore hiding of linked inputs in app mode#9677

Merged
AustinMroz merged 1 commit intocloud/1.41from
backport-9671-to-cloud-1.41
Mar 9, 2026
Merged

[backport cloud/1.41] Restore hiding of linked inputs in app mode#9677
AustinMroz merged 1 commit intocloud/1.41from
backport-9671-to-cloud-1.41

Conversation

@comfy-pr-bot
Copy link
Member

@comfy-pr-bot comfy-pr-bot commented Mar 9, 2026

Backport of #9671 to cloud/1.41

Automatically created by backport workflow.

┆Issue is synchronized with this Notion page by Unito

As a temporary fix for widgets being incorrectly hidden, #9669 allowed
all disabled widgets to be displayed.

This PR provides a more robust implementation to derive whether the
widget, as would be displayed from the root graph, is disabled.

Potential regression:
- Drag drop handlers are applied on node, not widgets. A subgraph
containing a "Load Image" node, does not allow dragging and dropping an
image onto the subgraphNode in order to load it. Because app mode
widgets would display from the original owning node prior to this PR,
these drag/drop handlers would apply. Placing "Load Image" nodes. I
believe this change makes behavior more consistent, but it warrants
consideration.

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-9671-Restore-hiding-of-linked-inputs-in-app-mode-31e6d73d365081688e37fbb931f3af68)
by [Unito](https://www.unito.io)
@comfy-pr-bot comfy-pr-bot added the backport Backporting a PR onto a release candidate label Mar 9, 2026
@comfy-pr-bot comfy-pr-bot requested a review from a team as a code owner March 9, 2026 20:26
@comfy-pr-bot comfy-pr-bot added the backport Backporting a PR onto a release candidate label Mar 9, 2026
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Mar 9, 2026
@github-actions
Copy link

github-actions bot commented Mar 9, 2026

🎨 Storybook: ✅ Built — View Storybook

Details

⏰ Completed at: 03/09/2026, 08:27:49 PM UTC

Links

@github-actions
Copy link

github-actions bot commented Mar 9, 2026

🎭 Playwright: ✅ 556 passed, 0 failed · 2 flaky

📊 Browser Reports
  • chromium: View Report (✅ 543 / ❌ 0 / ⚠️ 2 / ⏭️ 10)
  • chromium-2x: View Report (✅ 2 / ❌ 0 / ⚠️ 0 / ⏭️ 0)
  • chromium-0.5x: View Report (✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0)
  • mobile-chrome: View Report (✅ 10 / ❌ 0 / ⚠️ 0 / ⏭️ 0)

@github-actions
Copy link

github-actions bot commented Mar 9, 2026

⚡ Performance Report

No baseline found — showing absolute values.

Metric Value
canvas-idle: style recalcs 124
canvas-idle: layouts 0
canvas-idle: task duration 429ms
canvas-idle: heap delta -3.7 MB
canvas-mouse-sweep: style recalcs 183
canvas-mouse-sweep: layouts 12
canvas-mouse-sweep: task duration 1041ms
canvas-mouse-sweep: heap delta -3.0 MB
dom-widget-clipping: style recalcs 43
dom-widget-clipping: layouts 0
dom-widget-clipping: task duration 359ms
dom-widget-clipping: heap delta 7.4 MB
subgraph-dom-widget-clipping: style recalcs 73
subgraph-dom-widget-clipping: layouts 0
subgraph-dom-widget-clipping: task duration 420ms
subgraph-dom-widget-clipping: heap delta -826.3 KB
subgraph-idle: style recalcs 121
subgraph-idle: layouts 0
subgraph-idle: task duration 387ms
subgraph-idle: heap delta -3.2 MB
subgraph-mouse-sweep: style recalcs 169
subgraph-mouse-sweep: layouts 16
subgraph-mouse-sweep: task duration 916ms
subgraph-mouse-sweep: heap delta -5.2 MB
Raw data
{
  "timestamp": "2026-03-09T20:31:06.303Z",
  "gitSha": "93b1beeaa8e53251ad5cd487a98194496b0e5502",
  "branch": "backport-9671-to-cloud-1.41",
  "measurements": [
    {
      "name": "canvas-idle",
      "durationMs": 2038.5170000000699,
      "styleRecalcs": 123,
      "styleRecalcDurationMs": 19.768999999999995,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 424.99199999999996,
      "heapDeltaBytes": -3758832
    },
    {
      "name": "canvas-idle",
      "durationMs": 2029.7950000000355,
      "styleRecalcs": 124,
      "styleRecalcDurationMs": 21.868,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 392.3979999999999,
      "heapDeltaBytes": -3837228
    },
    {
      "name": "canvas-idle",
      "durationMs": 2030.6880000000547,
      "styleRecalcs": 124,
      "styleRecalcDurationMs": 28.441000000000003,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 468.78200000000004,
      "heapDeltaBytes": -4038920
    },
    {
      "name": "canvas-mouse-sweep",
      "durationMs": 2030.0029999999651,
      "styleRecalcs": 181,
      "styleRecalcDurationMs": 51.571,
      "layouts": 12,
      "layoutDurationMs": 3.2830000000000004,
      "taskDurationMs": 1038.853,
      "heapDeltaBytes": -3375432
    },
    {
      "name": "canvas-mouse-sweep",
      "durationMs": 2013.7690000000248,
      "styleRecalcs": 181,
      "styleRecalcDurationMs": 51.270999999999994,
      "layouts": 12,
      "layoutDurationMs": 3.345,
      "taskDurationMs": 992.7,
      "heapDeltaBytes": -3120304
    },
    {
      "name": "canvas-mouse-sweep",
      "durationMs": 2073.647000000051,
      "styleRecalcs": 186,
      "styleRecalcDurationMs": 58.151,
      "layouts": 12,
      "layoutDurationMs": 3.438,
      "taskDurationMs": 1090.3290000000002,
      "heapDeltaBytes": -2879888
    },
    {
      "name": "dom-widget-clipping",
      "durationMs": 563.0820000000085,
      "styleRecalcs": 41,
      "styleRecalcDurationMs": 12.434999999999999,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 346.978,
      "heapDeltaBytes": 7712232
    },
    {
      "name": "dom-widget-clipping",
      "durationMs": 604.7060000000783,
      "styleRecalcs": 46,
      "styleRecalcDurationMs": 16.85,
      "layouts": 1,
      "layoutDurationMs": 0.3080000000000001,
      "taskDurationMs": 377.65,
      "heapDeltaBytes": 8257048
    },
    {
      "name": "dom-widget-clipping",
      "durationMs": 565.7569999999623,
      "styleRecalcs": 42,
      "styleRecalcDurationMs": 12.497000000000002,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 351.87,
      "heapDeltaBytes": 7372592
    },
    {
      "name": "subgraph-dom-widget-clipping",
      "durationMs": 569.6990000000142,
      "styleRecalcs": 72,
      "styleRecalcDurationMs": 15.718000000000004,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 398.932,
      "heapDeltaBytes": 15282336
    },
    {
      "name": "subgraph-dom-widget-clipping",
      "durationMs": 633.6390000000165,
      "styleRecalcs": 75,
      "styleRecalcDurationMs": 16.097,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 437.26800000000003,
      "heapDeltaBytes": -8892884
    },
    {
      "name": "subgraph-dom-widget-clipping",
      "durationMs": 591.9509999999946,
      "styleRecalcs": 73,
      "styleRecalcDurationMs": 15.898,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 422.31699999999995,
      "heapDeltaBytes": -8927852
    },
    {
      "name": "subgraph-idle",
      "durationMs": 2004.8980000000256,
      "styleRecalcs": 121,
      "styleRecalcDurationMs": 20.935000000000002,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 390.052,
      "heapDeltaBytes": -3523104
    },
    {
      "name": "subgraph-idle",
      "durationMs": 2014.3470000000434,
      "styleRecalcs": 122,
      "styleRecalcDurationMs": 21.041,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 394.906,
      "heapDeltaBytes": -3218540
    },
    {
      "name": "subgraph-idle",
      "durationMs": 2006.7060000000083,
      "styleRecalcs": 121,
      "styleRecalcDurationMs": 20.584,
      "layouts": 0,
      "layoutDurationMs": 0,
      "taskDurationMs": 376.744,
      "heapDeltaBytes": -3472676
    },
    {
      "name": "subgraph-mouse-sweep",
      "durationMs": 1984.2650000000503,
      "styleRecalcs": 171,
      "styleRecalcDurationMs": 55.311,
      "layouts": 16,
      "layoutDurationMs": 4.724,
      "taskDurationMs": 980.5759999999999,
      "heapDeltaBytes": -5190480
    },
    {
      "name": "subgraph-mouse-sweep",
      "durationMs": 2073.167000000012,
      "styleRecalcs": 179,
      "styleRecalcDurationMs": 58.144,
      "layouts": 17,
      "layoutDurationMs": 4.622,
      "taskDurationMs": 1020.4000000000002,
      "heapDeltaBytes": -4506416
    },
    {
      "name": "subgraph-mouse-sweep",
      "durationMs": 1710.1490000000013,
      "styleRecalcs": 156,
      "styleRecalcDurationMs": 47.606,
      "layouts": 16,
      "layoutDurationMs": 4.769000000000001,
      "taskDurationMs": 747.8679999999999,
      "heapDeltaBytes": -6507868
    }
  ]
}

@AustinMroz AustinMroz merged commit ca66943 into cloud/1.41 Mar 9, 2026
44 checks passed
@AustinMroz AustinMroz deleted the backport-9671-to-cloud-1.41 branch March 9, 2026 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Backporting a PR onto a release candidate size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants